Skip to content

fix: ensure response format passed in aio. Ensure reconnecting ratherthan breaking. Correctly bubble up workflow name if set#951

Merged
sicoyle merged 7 commits intodapr:mainfrom
CasperGN:fix/handle-rst_stream
Mar 11, 2026
Merged

fix: ensure response format passed in aio. Ensure reconnecting ratherthan breaking. Correctly bubble up workflow name if set#951
sicoyle merged 7 commits intodapr:mainfrom
CasperGN:fix/handle-rst_stream

Conversation

@CasperGN
Copy link
Contributor

Description

Fixing bug where client would fail on RST_STREAM. Handle AIO responseformat & properly set wf name if name param passed.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

… than breaking. Correctly bubble up workflow name if set

Signed-off-by: Casper Nielsen <casper@diagrid.io>
@CasperGN CasperGN requested review from a team as code owners March 11, 2026 16:55
Copy link
Contributor

@sicoyle sicoyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

few comments pls

@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 89.79592% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.99%. Comparing base (bffb749) to head (874b816).
⚠️ Report is 89 commits behind head on main.

Files with missing lines Patch % Lines
...t/dapr-ext-workflow/tests/test_workflow_runtime.py 93.57% 14 Missing ⚠️
dapr/clients/grpc/client.py 0.00% 6 Missing ⚠️
dapr/clients/grpc/subscription.py 66.66% 3 Missing ⚠️
dapr/aio/clients/grpc/client.py 66.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #951      +/-   ##
==========================================
+ Coverage   86.63%   88.99%   +2.36%     
==========================================
  Files          84      103      +19     
  Lines        4473     7340    +2867     
==========================================
+ Hits         3875     6532    +2657     
- Misses        598      808     +210     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Signed-off-by: Casper Nielsen <casper@diagrid.io>
Copy link
Contributor

@sicoyle sicoyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@sicoyle sicoyle added this pull request to the merge queue Mar 11, 2026
Merged via the queue into dapr:main with commit 2fa367b Mar 11, 2026
23 of 26 checks passed
@dapr-bot
Copy link
Collaborator

The backport to release-1.17 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-release-1.17 release-1.17
# Navigate to the new working tree
cd .worktrees/backport-release-1.17
# Create a new branch
git switch --create backport-951-to-release-1.17
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 2fa367b6f02a3183a7339e5f85f296bb09492de6
# Push it to GitHub
git push --set-upstream origin backport-951-to-release-1.17
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-release-1.17

Then, create a pull request where the base branch is release-1.17 and the compare/head branch is backport-951-to-release-1.17.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants